Skip to content

[fix][model] Verified fix of openai error#209

Merged
lsy357 merged 1 commit into
coze-dev:mainfrom
panwangwin:fix/openai-error
Sep 25, 2025
Merged

[fix][model] Verified fix of openai error#209
lsy357 merged 1 commit into
coze-dev:mainfrom
panwangwin:fix/openai-error

Conversation

@panwangwin
Copy link
Copy Markdown
Contributor

@panwangwin panwangwin commented Sep 24, 2025 •
edited by lsy357
Loading

What type of PR is this?

fix

Check the PR title.

  • This PR title match the format: [<type>][<scope>]: <description>. For example: [fix][backend] flaky fix
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Add documentation if the current PR requires user awareness at the usage level.
  • This PR is written in English. PRs not in English will not be reviewed.

(Optional) Translate the PR title into Chinese.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):

(Optional) Which issue(s) this PR fixes:

#189 (comment)

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Sep 24, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@mocayo
Copy link
Copy Markdown
Collaborator

mocayo commented Sep 24, 2025

check pr title please

@lsy357
Copy link
Copy Markdown
Collaborator

lsy357 commented Sep 24, 2025

@yyyhhl Please help review this fix.

@panwangwin panwangwin changed the title Verified fix of openai error [Fix][Backend] Fix openai resource not find error Sep 24, 2025
@lsy357 lsy357 changed the title [Fix][Backend] Fix openai resource not find error [fix][model] Verified fix of openai error Sep 24, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 24, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #209      +/-   ##
==========================================
- Coverage   61.60%   61.60%   -0.01%     
==========================================
  Files         470      470              
  Lines       50017    50019       +2     
==========================================
  Hits        30813    30813              
- Misses      16934    16935       +1     
- Partials     2270     2271       +1
Flag Coverage Δ
unittests 61.60% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...nd/modules/llm/domain/service/llmimpl/eino/init.go 57.21% <100.00%> (+0.23%) ⬆️

... and 1 file with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e4b7973...8c9a08a. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yyyhhl yyyhhl self-requested a review September 25, 2025 07:01
@lsy357 lsy357 merged commit 271e206 into coze-dev:main Sep 25, 2025
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lsy357 lsy357 lsy357 approved these changes

@yyyhhl yyyhhl yyyhhl approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@panwangwin @CLAassistant @mocayo @lsy357 @yyyhhl